Skip to content

Make ms-python.python a soft dependency#123

Draft
aahrun wants to merge 2 commits intomainfrom
soft-python-dependency
Draft

Make ms-python.python a soft dependency#123
aahrun wants to merge 2 commits intomainfrom
soft-python-dependency

Conversation

@aahrun
Copy link
Copy Markdown
Collaborator

@aahrun aahrun commented Apr 3, 2026

  • Remove ms-python.python from extensionDependencies so the extension can install in VS Code derivatives (e.g. Cursor) whose marketplaces cannot resolve the dependency at install time.
  • Handle the missing Python extension gracefully at runtime: log a warning on startup and show a user-facing error when SDK discovery is attempted without it.

When ms-python.python is installed, behavior is unchanged.

- Remove `ms-python.python` from `extensionDependencies` so the extension can
  install in VS Code derivatives (e.g. Cursor) whose marketplaces cannot resolve
  the dependency at install time.
- Handle the missing Python extension gracefully at runtime: log a warning on
  startup and show a user-facing error when SDK discovery is attempted without it.

When `ms-python.python` is installed, behavior is unchanged.
@aahrun aahrun requested a review from a team as a code owner April 3, 2026 08:14
@aahrun aahrun closed this Apr 3, 2026
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 3, 2026
@aahrun aahrun reopened this May 4, 2026
@aahrun
Copy link
Copy Markdown
Collaborator Author

aahrun commented May 4, 2026

I initially created this as a potential fix to the issue we see when trying to install the mojo extension using the Cursor extension marketplace. I closed it hoping they'd have a fix in place soon, but potentially this is worth revisiting.

@aahrun aahrun marked this pull request as draft May 4, 2026 16:45
@aahrun
Copy link
Copy Markdown
Collaborator Author

aahrun commented May 4, 2026

The Python extension seems to come by default in a desktop install of VS Code, so making it a soft-dependency to potentially resolve the Cursor issue made sense, however it's not bundled in our test infrastructure, so can't get through CI.

Marked as a draft while I double check to see if the Cursor marketplace issue is resolved or not.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant